fix: CI enforcement for compile:context, skills index, and plugin registry tests#50
Merged
KailasMahavarkar merged 1 commit intomainfrom Apr 14, 2026
Merged
Conversation
…istry tests Critical issues from behaviour analysis: 1. CI: run compile:context before tests - bootstrap artifact was missing from repo (generated/ dir) - existing sync test would fail on fresh CI checkout - added compile step to publish.yml before bun test 2. CI: regenerate skills/INDEX.md before tests - index could drift silently when skills added/renamed - added generate-skills-index.sh step to publish.yml 3. tests/skills-index-behaviour.test.ts (new) - skills/INDEX.md stays in sync with actual skill dirs - every skill has required frontmatter (name, category, description) - every skill has valid category (core | domain | meta) - INDEX.md references every skill directory 4. tests/plugin-registry-behaviour.test.ts (new) - all 11 plugins register at least one tool - every tool has non-empty name and description - tool names follow namespace convention (plugin_name_action) - designer + shadcn register all tools referenced in skills - no duplicate tool names across plugins 29/29 tests pass
KailasMahavarkar
added a commit
that referenced
this pull request
Apr 14, 2026
…istry tests (#50) Critical issues from behaviour analysis: 1. CI: run compile:context before tests - bootstrap artifact was missing from repo (generated/ dir) - existing sync test would fail on fresh CI checkout - added compile step to publish.yml before bun test 2. CI: regenerate skills/INDEX.md before tests - index could drift silently when skills added/renamed - added generate-skills-index.sh step to publish.yml 3. tests/skills-index-behaviour.test.ts (new) - skills/INDEX.md stays in sync with actual skill dirs - every skill has required frontmatter (name, category, description) - every skill has valid category (core | domain | meta) - INDEX.md references every skill directory 4. tests/plugin-registry-behaviour.test.ts (new) - all 11 plugins register at least one tool - every tool has non-empty name and description - tool names follow namespace convention (plugin_name_action) - designer + shadcn register all tools referenced in skills - no duplicate tool names across plugins 29/29 tests pass
KailasMahavarkar
added a commit
that referenced
this pull request
Apr 14, 2026
…istry tests (#50) Critical issues from behaviour analysis: 1. CI: run compile:context before tests - bootstrap artifact was missing from repo (generated/ dir) - existing sync test would fail on fresh CI checkout - added compile step to publish.yml before bun test 2. CI: regenerate skills/INDEX.md before tests - index could drift silently when skills added/renamed - added generate-skills-index.sh step to publish.yml 3. tests/skills-index-behaviour.test.ts (new) - skills/INDEX.md stays in sync with actual skill dirs - every skill has required frontmatter (name, category, description) - every skill has valid category (core | domain | meta) - INDEX.md references every skill directory 4. tests/plugin-registry-behaviour.test.ts (new) - all 11 plugins register at least one tool - every tool has non-empty name and description - tool names follow namespace convention (plugin_name_action) - designer + shadcn register all tools referenced in skills - no duplicate tool names across plugins 29/29 tests pass
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes 3 critical issues found in behaviour analysis.
Critical Issues Fixed
1. Stale bootstrap artifact ships to CI
generated/runtime-context/using-hyperstack.bootstrap.mdwas missing from repobun run compile:contextstep topublish.ymlbefore tests2.
skills/INDEX.mddrifts silentlybash scripts/generate-skills-index.shstep topublish.ymlbefore tests3. Plugin tool regressions invisible
tests/plugin-registry-behaviour.test.tsNew Tests (29 total, all pass)
tests/skills-index-behaviour.test.tstests/plugin-registry-behaviour.test.tsplugin_name_action)